Skip to content

Fix figure visual parity with MATLAB toolbox#78

Merged
iahncajigas merged 3 commits intomainfrom
fix/figure-visual-parity
Mar 24, 2026
Merged

Fix figure visual parity with MATLAB toolbox#78
iahncajigas merged 3 commits intomainfrom
fix/figure-visual-parity

Conversation

@iahncajigas
Copy link
Copy Markdown
Contributor

Summary

  • Ex03 Fig05: Replace gray 3D wireframes with viridis pcolormesh heatmaps matching MATLAB's mesh() + view([90 -90])
  • Ex01 Figs01/03: Fix ACF panel scatter dot size for full-range visibility (matching MATLAB blue dots)
  • Ex02 Fig02: Add 95% CI error bars and significance stars to GLM coefficient plot
  • Ex05 Figs03/05/06: Change start/finish markers to unfilled (open) circles matching MATLAB
  • Ex03 Fig01: Add CIF legend with $\lambda_i$ label
  • LaTeX: Fix legend labels in Ex03 Fig06 for proper mathtext rendering

Test plan

  • All 25 figures regenerated and committed
  • 257 tests pass (3 expected skips)
  • Visual comparison confirms MATLAB parity for all fixed panels

🤖 Generated with Claude Code

Iahn Cajigas and others added 3 commits March 23, 2026 23:45
Example 03 Fig05 — stimulus effect surfaces:
  Replace 3D plot_wireframe (gray) with 2D pcolormesh (viridis colormap),
  matching MATLAB's mesh() with view([90 -90]) top-down heatmap rendering

Example 01 Figs 01/03 — plotInvGausTrans ACF panel:
  Reduce marker size from 4.0 to 3.0 so individual scatter dots are visible
  across the full lag range (~550 lags), matching MATLAB's dot-scatter style

Example 02 Fig02 — GLM coefficient error bars:
  Change error bars from 1x SE to 1.96x SE (95% CI) matching MATLAB;
  switch marker from "." to "o" at size 8 for visibility

Example 05 Figs 03/05/06 — start/finish markers:
  Change filled blue/red circles to unfilled (open) circles with
  markerfacecolor="none" and markeredgewidth=2, matching MATLAB

Example 03 Fig01 — CIF legend:
  Add label="$\lambda_i$" and legend(loc="upper right") to match MATLAB

LaTeX rendering:
  Fix legend labels in Example 03 Fig06 from unescaped strings to proper
  raw f-strings with $\lambda$ mathtext wrapping

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@iahncajigas iahncajigas merged commit 7105079 into main Mar 24, 2026
14 checks passed
@iahncajigas iahncajigas deleted the fix/figure-visual-parity branch March 24, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant